-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2.15.2 #94
2.15.2 #94
Conversation
Thanks for sending over the PR, @dwrusse!! Would you mind if I made some updates to this? Here's what I have in mind so that things flow with the rest of the functions:
I don't want to step on your toes and greatly appreciate you sending this over!! Let me know your feedback on it and let's use this to open a discussion up on things 😄 |
Thanks for reviewing @scrthq . All of your suggestions make sense and I agree with your reasoning, so no objections here. I can make the updates myself tomorrow if you'd like. I've also added the Set-GSGmailMessageLabel function if you'd like to include it in this version. |
Thanks @dwrusse! I'm actually almost done with the updates on my end, working off your forked branch now (this PR was actually good timing since I was just hopping on my dev env 😄). Wrapping in Color update support as well for full coverage of the available update fields Thanks for adding in the Set-GSGmailMessageLabel function as well!! I'll check it out next! 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @dwrusse!! Updates done, getting this merged ASAP! Cheers! 🎉
No description provided.